Skip to content

Remove index sync feature flag #6375

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
May 3, 2023

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Apr 24, 2023

This PR removes the FEATURE_INDEX_SYNC feature flag.

I've opened this as a draft PR since we just enabled the feature flag this morning and we should probably wait with the removal for 1-2 weeks until we are confident that enabling the flag did not unintentionally break anything.

@Turbo87 Turbo87 added C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear A-backend ⚙️ labels Apr 24, 2023
@Turbo87 Turbo87 force-pushed the remove-index-sync-feature-flag branch 3 times, most recently from 7bfaac3 to 6c771c6 Compare April 25, 2023 07:35
@bors
Copy link
Contributor

bors commented Apr 25, 2023

☔ The latest upstream changes (presumably #6380) made this pull request unmergeable. Please resolve the merge conflicts.

@Turbo87 Turbo87 force-pushed the remove-index-sync-feature-flag branch from 6c771c6 to aef34fe Compare April 25, 2023 09:07
@bors
Copy link
Contributor

bors commented Apr 26, 2023

☔ The latest upstream changes (presumably #6267) made this pull request unmergeable. Please resolve the merge conflicts.

@Turbo87 Turbo87 force-pushed the remove-index-sync-feature-flag branch from aef34fe to 92c9532 Compare April 26, 2023 09:58
@Turbo87 Turbo87 marked this pull request as ready for review May 3, 2023 08:18
Turbo87 added 5 commits May 3, 2023 10:18
This has been enabled in production now, so we can remove the feature flag.
This has been replaced by the two `SyncToGit/SparseIndex` jobs
This has been replaced by the two `SyncToGit/SparseIndex` jobs
This has been replaced by the two `SyncToGit/SparseIndex` jobs
@Turbo87 Turbo87 force-pushed the remove-index-sync-feature-flag branch from 92c9532 to cb22eb1 Compare May 3, 2023 08:18
@Turbo87 Turbo87 enabled auto-merge May 3, 2023 08:20
@Turbo87 Turbo87 merged commit ae120b4 into rust-lang:master May 3, 2023
@Turbo87 Turbo87 deleted the remove-index-sync-feature-flag branch May 3, 2023 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants